Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v1.0.0: transfer of functionalities for trait evolution from PhyloNetworks v0.16, with breaking changes and improvements #2

Merged
merged 52 commits into from
Jan 10, 2025

Conversation

cecileane
Copy link
Member

No description provided.

cecileane and others added 30 commits July 25, 2024 15:01
Moved getHeights getGammas and setGammas! to PN.

All tests should work except test_traits_discrete. It uses some functions that are in PhyLiNC
Since setLength! was removed from PN, places where edge lengths are changed are directly changed as opposed to using the function call.

Also made updateHybridRandomTrait! return true, making it work with the updated traversal functions.
Also brought back getGammas and setGammas.
Copy link

codecov bot commented Dec 24, 2024

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment

Thanks for integrating Codecov - We've got you covered ☂️

@jjustison
Copy link
Member

I just added a lightly modified version of the docs readme from the PN docs folder to note documentation practices and building things locally. There could potentially be an addition to this for the referencing external documentation and how the make.jl would be modified.

Otherwise I think this could be merged!

@jjustison
Copy link
Member

I added a small blurb about external references. I haven't actually used them but wrote this with my understanding from what I gathered from previous discussions and changes in the code

@cecileane cecileane requested a review from jjustison January 9, 2025 22:09
@jjustison jjustison merged commit cc7659b into main Jan 10, 2025
6 checks passed
@cecileane
Copy link
Member Author

fixes #1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants